Skip to content

[#146] Move New Story button to top of sidebar with accent styling#149

Merged
realproject7 merged 1 commit intomainfrom
task/146-new-story-button-top
Apr 23, 2026
Merged

[#146] Move New Story button to top of sidebar with accent styling#149
realproject7 merged 1 commit intomainfrom
task/146-new-story-button-top

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Fixes [UX] Move New Story button to top of sidebar with accent color #146 — New Story button was at the bottom and not visually prominent
  • Moved button to directly below the Stories header
  • Styled as primary action: accent background (bg-accent) with white text, hover dims
  • Updated empty state hint text to say "above" instead of "below"

Changes

  • app/web/components/StoryBrowser.tsx — Moved button markup, updated styling and hint text
  • app/web/dist/ — Rebuilt frontend
  • package.json — Version bump 1.0.10 → 1.0.11

Test plan

  • npm run typecheck — passes
  • npm run app:build — passes
  • Visual: button appears below header with brown accent bg and white text
  • Visual: button hover darkens correctly
  • Clicking still launches new story session

🤖 Generated with Claude Code

Repositioned + New Story from the bottom to directly below the Stories
header. Styled as primary action with accent background and white text.
Rebuilt frontend dist.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@realproject7
Copy link
Copy Markdown
Owner Author

Review — @re2

✅ APPROVED

Simple move: button relocated from bottom (border-t) to below header (border-b). Styling updated to bg-accent text-white hover:bg-accent-dim with centered text. Help text updated from "below" to "above". Old bottom button removed. Version 1.0.11 correct.

CI pending, no concerns.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: COMMENT

Summary

The UI change matches issue #146: the New Story button is now directly below the Stories header and uses accent background with white text.

Findings

  • No code blockers found in the updated diff.
  • lint-and-typecheck is still pending on current head commit, so build status is not yet confirmed.

Decision

No code blockers from re1. Holding final approval until current PR checks finish successfully.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The PR satisfies issue #146 by moving the New Story button directly below the Stories header and styling it as a primary accent action with white text.

Findings

  • No blocking findings.
  • lint-and-typecheck passes on the current head commit.

Decision

Approved. The change is focused, follows existing component patterns, and the required check is green.

@realproject7 realproject7 merged commit b4145fe into main Apr 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UX] Move New Story button to top of sidebar with accent color

2 participants